Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EUDI Interop #41

Merged
merged 6 commits into from
Mar 19, 2024
Merged

EUDI Interop #41

merged 6 commits into from
Mar 19, 2024

Conversation

nodh
Copy link
Contributor

@nodh nodh commented Mar 18, 2024

Work on interop with EUDI credentials

@nodh nodh requested a review from JesusMcCloud March 18, 2024 10:51
@nodh nodh self-assigned this Mar 18, 2024
@nodh nodh changed the title Draft: EUDI Interop EUDI Interop Mar 18, 2024
Copy link
Collaborator

@JesusMcCloud JesusMcCloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am unaware of any reals tests, to verify whether equality checks have as intended. If i overlooked them and we do, in fact have regression rests, please ignore my comments, otherwise, please provide tests

) {

fun serialize() = jsonSerializer.encodeToString(this)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where are the regression tests? Serious question here: do we know equality checks still work?

issuer = issuer,
issuanceDate = issuanceDate,
expirationDate = issuanceDate + lifetime,
credentialStatus = null,
credentialSubject = credentialSubject,
)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again: do we know equality checks still work?

@@ -38,40 +38,6 @@ data class VerifiableCredentialSdJwt(

fun serialize() = jsonSerializer.encodeToString(this)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

@@ -30,25 +30,4 @@ data class VerifiablePresentation(
audience = audienceId,
jwtId = id
)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

val revokedVerifiableCredentials: Array<VerifiableCredentialJws> = arrayOf(),
val invalidVerifiableCredentials: Array<String> = arrayOf(),
) {
override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

@@ -10,20 +10,5 @@ import kotlinx.serialization.Serializable
@Serializable
data class FormatContainerLdp(
@SerialName("proof_type")
val proofType: Array<String>,
) {
override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

constraints = constraints,
)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

formats: FormatHolder
) : this(id = uuid4().toString(), inputDescriptors = inputDescriptors, formats = formats)

fun serialize() = jsonSerializer.encodeToString(this)

override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

@@ -14,28 +14,5 @@ data class PresentationSubmission(
@SerialName("definition_id")
val definitionId: String,
@SerialName("descriptor_map")
val descriptorMap: Array<PresentationSubmissionDescriptor>? = null,
) {
override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

@@ -24,38 +24,5 @@ data class SubmissionRequirement(
@SerialName("from")
val from: String? = null,
@SerialName("from_nested")
val fromNested: Array<SubmissionRequirement>? = null,
) {
override fun equals(other: Any?): Boolean {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same same

@JesusMcCloud JesusMcCloud dismissed their stale review March 19, 2024 07:14

Make the comments into an issue

Copy link
Collaborator

@JesusMcCloud JesusMcCloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am adding an issue wrt. equality tests. Hence, I am leaving my comments here for reference, just that we won't miss anything.
I everything else looks good

@nodh nodh merged commit 2ce17ad into main Mar 19, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants